Skip to content

fix: bump prometheus to v0.311.2 for stored XSS CVE#223

Merged
notque merged 1 commit intomasterfrom
fix/prometheus-cve-xss
May 6, 2026
Merged

fix: bump prometheus to v0.311.2 for stored XSS CVE#223
notque merged 1 commit intomasterfrom
fix/prometheus-cve-xss

Conversation

@notque
Copy link
Copy Markdown
Contributor

@notque notque commented May 6, 2026

Summary

  • Bumps github.com/prometheus/prometheus from v0.304.1 → v0.311.2 to resolve dependabot alert Hierarchical projects #16 (stored XSS via metric names/label values in web UI tooltips, moderate severity)
  • Adapts pkg/util/promqlmod.go to new parser interface (package-level functions → methods on parser.NewParser())

Test plan

  • go build ./... passes
  • go test ./... — all packages pass
  • PromQL modification logic unchanged (same tests, same results)

Upgrades github.com/prometheus/prometheus from v0.304.1 to v0.311.2
(dependabot alert #16: stored XSS via metric names in web UI tooltips).

Adapts PromQL parser calls to new interface-based API — ParseExpr and
ParseMetricSelector are now methods on parser.NewParser() instead of
package-level functions.
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

Merging this branch will not change overall coverage

Impacted Packages Coverage Δ 🤖
github.com/SAP-cloud-infrastructure/maia/pkg/util 83.33% (ø)

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/SAP-cloud-infrastructure/maia/pkg/util/promqlmod.go 87.88% (ø) 165 145 20

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

@notque notque merged commit 20d055e into master May 6, 2026
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants